-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix ORC related failed UT #417
Conversation
@rui-mo could you have a look for this PR |
Thanks. Can we open this change to Velox upstream? We can pick them back to this branch. |
@chenxu14 Thanks, I see. Maybe we can keep this PR in our local repo to get ORC work with minimal change. |
84b0fea
to
47ed824
Compare
34187ee
to
4032e68
Compare
c7837d9
to
dfd7071
Compare
@chenxu14 We are frequently rebasing the base branch. Could you rebase this PR? |
It looks like the diff is wrong as we force-push to update branch, the workable commit is this one: |
4d877e0
to
f68b4c0
Compare
e3d9542
to
17644f8
Compare
f8e0c6d
to
8d8b7d5
Compare
No description provided.